Skip to content

Conditional report actions#276

Draft
TheSyscall wants to merge 4 commits into
mainfrom
conditional-actions
Draft

Conditional report actions#276
TheSyscall wants to merge 4 commits into
mainfrom
conditional-actions

Conversation

@TheSyscall

@TheSyscall TheSyscall commented Apr 16, 2026

Copy link
Copy Markdown
Contributor

Only show actions if there is actually an exporter for it.

Draft for now because of an issue that prevents PrintableHtmlDocument to be moved to icingaweb Icinga/icingaweb2#5510

  • Remove PDF download option and mail attachment type when there is no pdfexport hook implementation registered
  • Remove download dropdown and send button entirely if no export option is available

This serves as a step toward getting rid of pdfexport as a requirement for reporting, but does not get rid of this dependency entirely.

This combined with the changes made in #275 should allow us to remove pdfexport as a requirement. It is still not possible to use any module that implements the Pdfexport hook but at least the module is able to show reports in the web app and export to CSV and JSON.

@cla-bot cla-bot Bot added the cla/signed label Apr 16, 2026
@TheSyscall TheSyscall requested a review from Al2Klimov April 16, 2026 12:15
Comment thread library/Reporting/Actions/SendMail.php Outdated
Comment thread application/controllers/ReportController.php Outdated
Comment thread application/controllers/ReportController.php
@TheSyscall TheSyscall requested a review from Al2Klimov June 22, 2026 07:25
Al2Klimov

This comment was marked as resolved.

@TheSyscall TheSyscall requested a review from Al2Klimov June 23, 2026 13:56
@TheSyscall TheSyscall marked this pull request as draft June 25, 2026 08:33
@TheSyscall TheSyscall self-assigned this Jun 25, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants